Skip to content

Add Room Average Temperature sensor to Lyric integration - #177063

Merged
joostlek merged 1 commit into
home-assistant:devfrom
clutch2sft:sensor-room-average-temperature
Jul 29, 2026
Merged

Add Room Average Temperature sensor to Lyric integration#177063
joostlek merged 1 commit into
home-assistant:devfrom
clutch2sft:sensor-room-average-temperature

Conversation

@clutch2sft

@clutch2sft clutch2sft commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Resideo's priority endpoint reports a room-level average temperature (roomAvgTemp) alongside the per-accessory temperature already exposed as Room Temperature. This was parsed but never surfaced. Adds it as a sibling entry to the existing Room Humidity sensor, which already uses the same room-level (not accessory-level) value.

Proposed change

Resideo's priority endpoint reports a room-level average temperature
(roomAvgTemp/avgTemperature) alongside the per-accessory temperature already
exposed as Room Temperature. This was parsed but never surfaced. Adds it as a
sibling entry to the existing Room Humidity sensor, which already uses the same
room-level (not accessory-level) value.

Dependency note: this needs both #177022 (coordinator actually fetching room data)
and the pending aiolyric field-name fix (room_avg_temp currently reads the wrong
JSON key) to show real data. Safe to merge independently — it'll just report
unavailable until both land, same as the existing Room Humidity sensor already
does today.

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New integration (thank you!)
  • [x ] New feature (which adds functionality to an existing integration)
  • Deprecation (breaking change to happen in the future)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Additional information

Checklist

  • [ x] I understand the code I am submitting and can explain how it works.
  • [ x] The code change is tested and works locally.
  • [ x] Local tests pass. Your PR cannot be merged unless tests pass
  • [x ] There is no commented out code in this PR.
  • [ x] I have followed the development checklist
  • [x ] I have followed the perfect PR recommendations
  • [ x] The code has been formatted using Ruff (ruff format homeassistant tests)
  • Tests have been added to verify that the new code works.
  • [ x] Any generated code has been carefully reviewed for correctness and compliance with project standards.

If user exposed functionality or configuration variables are added/changed:

If the code communicates with devices, web services, or third-party tools:

  • The manifest file has all fields filled out correctly.
    Updated and included derived files by running: python3 -m script.hassfest.
  • New or updated dependencies have been added to requirements_all.txt.
    Updated by running python3 -m script.gen_requirements_all.
  • For the updated dependencies a diff between library versions and ideally a link to the changelog/release notes is added to the PR description.

To help with the load of incoming pull requests:

Resideo's priority endpoint reports a room-level average temperature
(roomAvgTemp) alongside the per-accessory temperature already
exposed as Room Temperature. This was parsed but never surfaced.
Adds it as a sibling entry to the existing Room Humidity sensor,
which already uses the same room-level (not accessory-level) value.
Copilot AI review requested due to automatic review settings July 22, 2026 09:42
@home-assistant

Copy link
Copy Markdown
Contributor

Hey there @timmo001, mind taking a look at this pull request as it has been labeled with an integration (lyric) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of lyric can trigger bot actions by commenting:

  • @home-assistant close Closes the pull request.
  • @home-assistant mark-draft Mark the pull request as draft.
  • @home-assistant ready-for-review Remove the draft status from the pull request.
  • @home-assistant rename Awesome new title Renames the pull request.
  • @home-assistant reopen Reopen the pull request.
  • @home-assistant unassign lyric Removes the current integration label and assignees on the pull request, add the integration domain after the command.
  • @home-assistant update-branch Update the pull request branch with the base branch.
  • @home-assistant add-label needs-more-information Add a label (needs-more-information, problem in dependency, problem in custom component, problem in config, problem in device, feature-request) to the pull request.
  • @home-assistant remove-label needs-more-information Remove a label (needs-more-information, problem in dependency, problem in custom component, problem in config, problem in device, feature-request) on the pull request.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a room-average temperature sensor to the Lyric integration.

Changes:

  • Adds a temperature sensor backed by room_avg_temp.
  • Adds the corresponding entity translation.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
homeassistant/components/lyric/sensor.py Defines the room-average temperature sensor.
homeassistant/components/lyric/strings.json Adds its display name.

translation_key="room_average_temperature",
device_class=SensorDeviceClass.TEMPERATURE,
state_class=SensorStateClass.MEASUREMENT,
value_fn=lambda room, _: room.room_avg_temp,
value_fn=lambda room, _: room.room_avg_humidity,
suitable_fn=lambda _, accessory: accessory.type == "IndoorAirSensor",
),
LyricSensorAccessoryEntityDescription(
Comment on lines +64 to +66
"room_average_temperature": {
"name": "Room average temperature"
},

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Documentation added/updated for www.home-assistant.io
Link to documentation pull request: home-assistant/home-assistant.io#46986

@joostlek
joostlek merged commit beb0a8a into home-assistant:dev Jul 29, 2026
33 checks passed
raman325 added a commit to raman325/home-assistant that referenced this pull request Jul 29, 2026
* upstream/dev: (1263 commits)
  Fix Schlage test snapshot (home-assistant#177548)
  Clarify common config flow already_in_progress abort message (home-assistant#176981)
  Add more button entities to LED Infrared (home-assistant#177400)
  Promote energieleser integration to platinum quality scale (home-assistant#176617)
  Fix UnboundLocalError in state condition with an empty state value (home-assistant#176634)
  Add async_will_remove_from_hass to pylint super call checker (home-assistant#175986)
  Add Room Average Temperature sensor to Lyric integration (home-assistant#177063)
  Bump github/codeql-action/analyze from 4.37.2 to 4.37.3 (home-assistant#177524)
  Update ruff to 0.16.0 (home-assistant#177534)
  Add restore state to vacuum template entities (home-assistant#176718)
  Add restore to lock template entities (home-assistant#176571)
  Add tilt support to MySensors cover platform (home-assistant#173845)
  Add restore to light template entities (home-assistant#176553)
  Add zoned cleaning service for Roborock (home-assistant#172673)
  Add power control to Daikin zone climates (home-assistant#176764)
  Add image platform for DoorBird last motion and last ring (home-assistant#173120)
  Explain why a Comet Blue device could not be found (home-assistant#176768)
  Add ViAir 300F humidity and temperature sensors to Vicare (home-assistant#176777)
  Schlage: refactor tests to use snapshot_platform and factory fixture (home-assistant#172639)
  Undeprecate http server_host configuration (home-assistant#177536)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants